Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

rest: match paths using regexp #79

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

mgenov
Copy link
Contributor

@mgenov mgenov commented Jan 21, 2016

Adds regex pattern matching on paths, so path variables could be more
complex that are now.

Here are some examples

/r/users/:id{\d+}

will match only

/r/users/123

but not

/r/users/john

Note: testng version was updated in pom.xml due issue with latest Intellij IDEA 15. If this change is breakable, the pom.xml changes will be reverted.

Adds regex pattern matching on paths, so path variables could be more
complex that are now.

Here are some examples
```
/r/users/:id{\d+}
```

will match only

```
/r/users/123
```

but not

```
/r/users/john
```
@mgenov
Copy link
Contributor Author

mgenov commented Jan 21, 2016

cc @jvanzyl @dhanji

@mgenov
Copy link
Contributor Author

mgenov commented Jan 21, 2016

Another possible option would be brackets to wrap variable and regexp: {id:\d+}

@mgenov
Copy link
Contributor Author

mgenov commented Feb 13, 2016

ping

@mgenov
Copy link
Contributor Author

mgenov commented May 5, 2016

ping2

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant